Fix typo in RemindingNotifier interval setter#5397
Conversation
Signed-off-by: Vinod Kumar <codingkiddo@gmail.com>
|
I'm not a mantainer of the project, so my opinion here counts very little, but the fix is so trivial for whoever is using the feature that I wouldn't bother to keep the mistyped method, to be honest. |
Thanks for taking a look. I kept the misspelled setter as a deprecated forwarder because this is a public API method and removing it could break existing users. I’m happy to remove the compatibility method if the maintainers prefer a direct rename only. |
Understandable and, depending on the type of the API and the effort to migrate to the new one, I would have potentially done the same. |
That’s a fair concern. If we keep the deprecated forwarder, I’d suggest treating it as a compatibility bridge for the 4.x line and removing it in the next major version, 5.0.0, rather than 4.1.0. Since the misspelled method is already part of the public API, removing it in a minor release could still surprise users who call it directly. That said, I’m also fine to simplify this PR and remove the deprecated method if the maintainers prefer not to carry the compatibility API for this typo. |
|
Up to the team ;) |
Fixes a typo in
RemindingNotifier's check reminder interval setter.The existing public method name
setCheckReminderInvervalcontains a typo. This change adds the correctly spelledsetCheckReminderIntervalmethod and keeps the existing misspelled method as a deprecated forwarder for backward compatibility.Also updated sample configuration, documentation examples, and tests to use the corrected setter name.
Tests run:
mvn -pl spring-boot-admin-server -Dtest=RemindingNotifierTest testgit diff --check